Skip to content

Test registry cache - do not merge - #9049

Closed
nolanpro wants to merge 1 commit into
developfrom
test-registry-cache
Closed

nolanpro wants to merge 1 commit into
developfrom
test-registry-cache

Conversation

@nolanpro

@nolanpro nolanpro commented Sep 8, 2026 •

Copy link
Copy Markdown
Contributor

ci:k8s-branch:test-registry-cache
ci:deploy

ci:run-testbench

@cursor

cursor Bot commented Sep 8, 2026 •

Copy link
Copy Markdown

PR Summary

Medium Risk
Switching the reusable workflow from @main to @js-builds-2 changes CI build/deploy for every trigger of this workflow; unintended merge could destabilize or diverge from standard pipelines until reverted.

Overview
What changed

The deploy-pm4 workflow no longer calls the shared deploy-pm4.yml workflow at @main; it now references @js-builds-2, so all runs of this workflow use that branch’s CI behavior (aligned with the PR’s experimental ci:k8s-branch:test-registry-cache intent).

README.md gains a single stray . as the first line before the existing heading—likely accidental noise rather than real documentation.

Review focus

Confirm that pinning to js-builds-2 is intentional and temporary, and whether the README leading . should be dropped before merge.

Reviewed by Cursor Bugbot for commit cef3148. Bugbot is set up for automated code reviews on this repo. Configure here.

@nolanpro

Copy link
Copy Markdown
Contributor Author

QA server K8S was successfully deployed https://ci-df5ad1fe74.engk8s.processmaker.net

@decisions-sonarqube

Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@nolanpro

Copy link
Copy Markdown
Contributor Author

QA server K8S was successfully deployed https://ci-df5ad1fe74.engk8s.processmaker.net

Co-authored-by: Cursor <cursoragent@cursor.com>
@nolanpro
nolanpro force-pushed the test-registry-cache branch 2 times, most recently from cef3148 to 35f50b5 Compare November 6, 2026 20:38
@nolanpro nolanpro changed the title Test build - do not merge Test registry cache - do not merge Nov 6, 2026
@nolanpro

nolanpro commented Nov 7, 2026

Copy link
Copy Markdown
Contributor Author

QA server K8S was successfully deployed https://tenant-1.ci-df5ad1fe74.engk8s.processmaker.net

1 similar comment
@nolanpro

nolanpro commented Nov 7, 2026

Copy link
Copy Markdown
Contributor Author

QA server K8S was successfully deployed https://tenant-1.ci-df5ad1fe74.engk8s.processmaker.net

@decisions-sonarqube

Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@nolanpro nolanpro closed this Nov 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant